chore: maak het pakket aantoonbaar Laravel 13 proof - #66
Merged
Conversation
Het weghalen van de brick/math pin maakte entrust installeerbaar naast Laravel 13, maar niemand kon zien of het er ook op werkte. Deze commit regelt dat. CI verifieerde niets. De workflow triggerde op master terwijl de default branch main heet, en de teststap stond uitgecommentarieerd. Wat er draaide was composer validate plus een install. Nu draait hij op main, met een matrix over Laravel 12 en 13, en roept hij het test-script aan dat er nog niet was. minimum-stability stond op dev met prefer-stable false. Daardoor resolveerde alles naar dev-branches: illuminate/support 13.x-dev, phpunit 12.5.x-dev en mockery 2.0.x-dev. Die laatste alleen al gaf 44 deprecations. Nu stable met prefer-stable true, en mockery ^1.6 en rector ^2.0 in plaats van versies die geen stable release hebben. De require-lijst was een platgeslagen dependency-tree van 47 pakketten waarvan er 39 transitief zijn. Dat is waar de brick/math pin vandaan kwam, en zonder opschonen komt de volgende blocker er bij Laravel 14 gewoon weer in. Teruggebracht tot de acht illuminate-pakketten die de source echt aanraakt: cache voor TaggableStore, console voor Command, contracts voor Auth\Guard, database voor Model, Migration, Blueprint en BelongsToMany, filesystem en view voor de migration-generator, http voor Request in de middleware, en support voor de facades, ServiceProvider en Str. Die pakketten stonden op ^13.0, wat betekende dat 2.5.8 Laravel 13 afdwong. Nu ^12.0|^13.0, zodat een tag kan landen zonder dat afnemers gelijktijdig mee moeten upgraden. Tot slot had makeFilterName() in de tests een impliciet nullable parameter, goed voor de laatste overgebleven deprecation. Getest op beide: 46 tests, 95 assertions, geen deprecations, phplint schoon op illuminate/support v12.65.0 en v13.24.0. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Het branches-filter op pull_request wordt door GitHub gewogen tegen de kopie van de workflow op de base branch, en daar staat nog master. De matrix draaide daardoor niet op deze PR. Filter eraf; push blijft beperkt tot main. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Vervolg op #65. Die haalde de
brick/mathpin weg zodat entrust naast Laravel 13 te installeren is. Bij het controleren of het er daarna ook echt op draait bleek dat niet vast te stellen, want de CI verifieerde niets. Deze PR maakt dat wel aantoonbaar.Wat er mis was
De CI verifieerde niets. De workflow triggerde op
masterterwijl de default branchmainheet, en de teststap stond uitgecommentarieerd:Een groene build betekende dus alleen dat
composer validateencomposer installslaagden. Er was ook geentest-script om aan te roepen.Alles resolveerde naar dev-branches.
minimum-stability: devmetprefer-stable: falseleverdeilluminate/support 13.x-dev,phpunit 12.5.x-devenmockery 2.0.x-devop. Die laatste is niet stable uitgebracht en gaf in zijn eentje 44 deprecations. Hetzelfde gold voorrector/rector: dev-main.De require-lijst was een platgeslagen dependency-tree. 47 pakketten waarvan er 39 transitief zijn:
psr/*,symfony/polyfill-*,symfony/cache,symfony/options-resolver,voku/portable-ascii,nesbot/carbon,nunomaduro/termwinden zo verder. Daar kwam debrick/mathpin vandaan. Dat is geen incident: zolang die lijst er staat komt er bij Laravel 14 gewoon een nieuwe blocker in.illuminate/* ^13.0dwong Laravel 13 af. Sinds 2.5.8 kon je entrust niet meer op Laravel 12 installeren, wat een upgrade van een afnemer tot een alles-tegelijk-operatie maakt.Wat er nu staat
De require is teruggebracht tot de acht illuminate-pakketten die de source daadwerkelijk aanraakt, opgezocht via de
use-statements insrc/:illuminate/cacheTaggableStoreilluminate/consoleCommandin de migration-generatorilluminate/contractsAuth\Guardilluminate/databaseModel,Migration,Blueprint,BelongsToManyilluminate/filesystemilluminate/httpRequestin de middlewareilluminate/supportBlade,Cache,Config,Response,Facade,ServiceProvider,Strilluminate/viewentrust::generators.migrationnamespaceAlle acht op
^12.0|^13.0. Verderminimum-stability: stablemetprefer-stable: true,mockery ^1.6enrector ^2.0, en eentest- pluslint-script.De workflow draait nu op
mainmet een matrix over beide Laravel-versies, pint die per job metcomposer require --no-updateen draait daarna echt de tests.Eén regel testcode mee:
makeFilterName()hadarray $permissions = nullzonder vraagteken, goed voor de laatste overgebleven deprecation.Getest
Lokaal op PHP 8.4.22, beide takken van de matrix:
composer validate --strictOok gecontroleerd aan de afnemerskant: parnassys-schoolkassa-backend resolveert met deze versie zowel op
laravel/framework v12.65.0als opv13.24.0.Gerebased op de huidige
main, dus de dependabot-bumps vanactions/checkout@v7enactions/cache@v6blijven staan.Na merge is er nog steeds een tag
2.5.9nodig.mainis inmiddels wel voorbij tag2.5.8gelopen, maar zonder tag ziet Packagist geen van deze wijzigingen.🤖 Generated with Claude Code